src/main/java/com/campustable/be/domain/cart/dto/CartRequest.java (1)

7-11: 요청 DTO에 입력 검증 어노테이션 추가를 권장합니다.

menuId가 null이거나 quantity가 음수일 경우 예기치 않은 동작이 발생할 수 있습니다. Bean Validation 어노테이션을 추가하면 서비스 레이어 진입 전에 잘못된 입력을 걸러낼 수 있습니다.

🔎 검증 어노테이션 추가 제안

 package com.campustable.be.domain.cart.dto;
 
+import jakarta.validation.constraints.Min;
+import jakarta.validation.constraints.NotNull;
 import lombok.Getter;
 
 @Getter
 public class CartRequest {
 
+  @NotNull(message = "메뉴 ID는 필수입니다")
   private Long menuId;
+  @Min(value = 0, message = "수량은 0 이상이어야 합니다")
   private int quantity;
 }

src/main/java/com/campustable/be/domain/cart/entity/Cart.java (1)

38-40: 양방향 관계 동기화를 위한 편의 메서드 추가를 권장합니다.

생성자에서 user.setCart(this)를 호출하지 않으면, User 측에서 getCart()를 호출할 때 영속성 컨텍스트가 갱신되지 않아 불일치가 발생할 수 있습니다.

🔎 양방향 관계 동기화 제안

   public Cart(User user) {
     this.user = user;
+    user.setCart(this);
   }

src/main/java/com/campustable/be/global/common/SecurityUtil.java (1)

18-30: 에러 코드 의미론적 불일치

인증 정보가 없거나 유효하지 않은 경우 USER_NOT_FOUND를 사용하는 것은 의미상 맞지 않습니다. 이 경우는 "유저를 찾을 수 없음"이 아니라 "인증되지 않은 요청"에 해당합니다.

ACCESS_DENIED 또는 새로운 UNAUTHORIZED 에러 코드를 사용하는 것이 더 적절합니다. 이렇게 하면 클라이언트가 에러 원인을 정확히 파악하고 적절한 대응(재로그인 등)을 할 수 있습니다.

🔎 수정 제안

    if (authentication == null || authentication.getName() == null || authentication.getName().isBlank()) {
      log.error("SecurityUtil class에서 시큐리티 컨텍스트 홀더에 유저가 존재하지않아 에러발생.");
-      throw new CustomException(ErrorCode.USER_NOT_FOUND);
+      throw new CustomException(ErrorCode.ACCESS_DENIED);
    }

    try {
      return Long.parseLong(authentication.getName());
    } catch (NumberFormatException e) {
-      throw new CustomException(ErrorCode.USER_NOT_FOUND);
+      log.error("인증 정보의 사용자 ID 형식이 올바르지 않습니다: {}", authentication.getName());
+      throw new CustomException(ErrorCode.ACCESS_DENIED);
    }

src/main/java/com/campustable/be/domain/cart/controller/CartController.java (3)

30-34: 요청 DTO 유효성 검증 누락

CartRequest에 @Valid 어노테이션이 없습니다. Bean Validation을 사용하려면 @Valid를 추가해야 합니다.

🔎 수정 제안

+import jakarta.validation.Valid;
...
  @PostMapping("/items")
-  public ResponseEntity<CartResponse> addOrUpdateItems(@RequestBody CartRequest request) {
+  public ResponseEntity<CartResponse> addOrUpdateItems(@Valid @RequestBody CartRequest request) {

---

46-52: 삭제 API 응답 코드 개선 권장

void 반환 대신 ResponseEntity를 반환하여 적절한 HTTP 상태 코드(예: 204 No Content)를 명시하는 것이 RESTful API 설계에 더 적합합니다.

또한, URL 경로에 cartId를 받는 대신 현재 인증된 사용자의 장바구니를 자동으로 조회하는 방식이 더 안전합니다. 현재 구조에서는 서비스 레이어에서 소유권 검증이 반드시 수행되어야 합니다.

🔎 수정 제안

  @DeleteMapping("/{cartId}")
-  public void deleteCart(@PathVariable Long cartId) {
+  public ResponseEntity<Void> deleteCart(@PathVariable Long cartId) {
    cartService.deleteCart(cartId);
+    return ResponseEntity.noContent().build();
  }

또는 현재 사용자의 장바구니를 자동으로 조회하는 방식:

  @DeleteMapping("")
  public ResponseEntity<Void> deleteCart() {
    cartService.deleteCurrentUserCart();
    return ResponseEntity.noContent().build();
  }

서비스 레이어에서 deleteCart(Long cartId) 호출 시 현재 사용자가 해당 장바구니의 소유자인지 검증하는 로직이 있는지 확인이 필요합니다.

---

54-59: deleteCartItem도 동일하게 ResponseEntity 반환 권장

🔎 수정 제안

  @DeleteMapping("/items/{cartItemId}")
-  public void deleteCartItem(@PathVariable Long cartItemId) {
+  public ResponseEntity<Void> deleteCartItem(@PathVariable Long cartItemId) {
    cartService.deleteCartItem(cartItemId);
+    return ResponseEntity.noContent().build();
  }

src/main/java/com/campustable/be/domain/cart/entity/CartItem.java (2)

22-25: 생성자 위치를 필드 선언 이후로 이동하는 것을 고려하세요.

생성자가 필드 선언보다 먼저 정의되어 있습니다. 일반적인 Java 코딩 관례는 필드 선언, 생성자, 메서드 순서입니다. 가독성 향상을 위해 생성자를 필드 선언 이후로 이동하는 것을 권장합니다.

---

31-35: @joincolumn 애너테이션을 명시하여 외래 키 컬럼명을 명확히 하세요.

@ManyToOne 관계에 @JoinColumn 애너테이션이 누락되어 있습니다. JPA가 기본 컬럼명을 생성하지만, 명시적으로 지정하면 스키마가 더 명확해지고 유지보수가 용이해집니다.

🔎 제안하는 수정

 @ManyToOne(fetch = FetchType.LAZY)
+@JoinColumn(name = "menu_id")
 private Menu menu;

 @ManyToOne(fetch = FetchType.LAZY)
+@JoinColumn(name = "cart_id")
 private Cart cart;

src/main/java/com/campustable/be/domain/cart/service/CartService.java (2)

56-67: 조건문에 중괄호를 추가하여 가독성과 안전성을 높이세요.

Line 60의 else 문에 중괄호가 없습니다. 향후 코드 수정 시 의도하지 않은 버그가 발생할 수 있으므로, 단일 문장이더라도 중괄호를 사용하는 것이 좋습니다.

🔎 제안하는 수정

     if (cartItemOpt.isPresent()) {
       if (quantity == 0){
         cartItemRepository.delete(cartItemOpt.get());
       }
-      else cartItemOpt.get().setQuantity(quantity);
+      else {
+        cartItemOpt.get().setQuantity(quantity);
+      }
     } else {
       if (quantity > 0) {
         CartItem newItem = new CartItem(cart, menu);
         newItem.setQuantity(quantity);
         cartItemRepository.save(newItem);
       }
     }

---

71-86: 중복 코드를 헬퍼 메서드로 추출하세요.

CartItemDto 리스트 생성(lines 71-73)과 총 가격 계산(lines 84-86) 로직이 getCart() 메서드(lines 146-148, 150-152)에서도 동일하게 반복됩니다. 이를 private 헬퍼 메서드로 추출하면 코드 유지보수성이 향상됩니다.

🔎 제안하는 리팩토링

클래스에 다음 헬퍼 메서드들을 추가:

private List<CartItemDto> mapToCartItemDtos(Cart cart) {
  return cartItemRepository.findByCart(cart).stream()
      .map(CartItemDto::new)
      .toList();
}

private int calculateTotalPrice(List<CartItemDto> cartItems) {
  return cartItems.stream()
      .mapToInt(item -> item.getPrice() * item.getQuantity())
      .sum();
}

그런 다음 중복 코드를 다음과 같이 교체:

-    List<CartItemDto> cartItems = cartItemRepository.findByCart(cart).stream().
-        map(CartItemDto::new).
-        toList();
+    List<CartItemDto> cartItems = mapToCartItemDtos(cart);

     if (cartItems.isEmpty()) {
       // ...
     }

-    int totalPrice = cartItems.stream()
-        .mapToInt(item->item.getPrice() * item.getQuantity())
-        .sum();
+    int totalPrice = calculateTotalPrice(cartItems);

getCart() 메서드에도 동일하게 적용하세요.

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro